Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

uncrustify: update config file and run uncrustify #183

Merged
merged 6 commits into from
Jun 5, 2024

Conversation

marckleinebudde
Copy link
Contributor

@marckleinebudde marckleinebudde commented Jun 4, 2024

Upgrade uncrustify's config file to the latest version.
Then uncrustify the header files in include.
Also I've found some unused defines in the config.h, remove them.
Remove comments about not connected silent pin and define CONFIG_CANFD to 1, this way uncrustify produces a nice looking config.h.

Fixes: 89c44fe ("correct LED initialization for canable interfaces")
This way uncrustify produces better indention.
This way uncrustify produces better indention.
@fenugrec
Copy link
Collaborator

fenugrec commented Jun 5, 2024

awesome. Let me know when it's ready (will probably do this, then 182, then 139, unless you have a different preferred order ?)

@marckleinebudde
Copy link
Contributor Author

This PR is ready, please pull it first, followed by #182. I'm still working on the multichannel stuff.

@fenugrec fenugrec merged commit eab3d53 into candle-usb:master Jun 5, 2024
2 checks passed
@fenugrec
Copy link
Collaborator

fenugrec commented Jun 5, 2024

Merged, thanks ! tested with uncrustify 0.77

@marckleinebudde marckleinebudde deleted the uncrustify branch June 6, 2024 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants